Skip to content

Note down text of a potential errata report about schema-node-path. - #171

Open
cabo wants to merge 1 commit into
masterfrom
errata-sid
Open

Note down text of a potential errata report about schema-node-path.#171
cabo wants to merge 1 commit into
masterfrom
errata-sid

Conversation

@cabo

@cabo cabo commented Mar 9, 2026

Copy link
Copy Markdown
Member

No description provided.

@cabo
cabo requested a review from abierman March 9, 2026 09:01
@EskoDijk

Copy link
Copy Markdown
Collaborator

I wasn't aware of this PR yet, and would like to review! Can I be added as a reviewer?

@EskoDijk
EskoDijk self-requested a review March 20, 2026 11:38

Note that both changes simply add a single sentence at the end of the description.

(Should there be a change to "revision 2024-07-31"?)

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Probably.


Note that both changes simply add a single sentence at the end of the description.

(Should there be a change to "revision 2024-07-31"?)

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes, any changes lead to a new YANG module revision. And each revision needs an additional 'revision' statement that's added on top of the existing one. So it would look like e.g.:

revision 2026-03-20 {
    description
      "Updates per errata report NNN.";
    reference
      "Errata report NNN <link>";
 }

revision 2024-07-31 {
    description
      "Initial revision.";
    reference
      "RFC 9595: YANG Schema Item iDentifier (YANG SID)";
  }

Also the new module needs to be manually submitted to the IETF YANG modules repository, in case we don't have a draft from which the YANG module gets auto-extracted by the tools.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

goog catch/

This is a new procedure
IANA will maintain new YANG revisions with Errata.
The link to this new module should be on the RFC page I guess. Not just Errata Exists.
tooling challenge: how to propagate IANA updates like this?

@vvilimek

Copy link
Copy Markdown
Collaborator

I would also include a definition of the term "NID". I am not sure if we want to rewrite the text to use it, but proper definition might be handy. Even if it put it on some GitHub core-wg Wiki or GitHub Pages.

The term is repeatedly used during discussion which may confuse newcomers.

@vvilimek

Copy link
Copy Markdown
Collaborator

Shouldn't the typedef schema-node-path be renamed to something like nid-path or node-item-identifier-path?
I think we also should define the term NID.

@vvilimek

Copy link
Copy Markdown
Collaborator

Related issue #177, I am not sure how to correct identifier form of rc:yang-data should look like. Similarly for sx:structure.

@vvilimek

vvilimek commented Jun 3, 2026

Copy link
Copy Markdown
Collaborator

Please, firstly resolve the #181

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants